Skip to content

Unify _to_date converter and add git workflow guidelines#641

Merged
laughingman7743 merged 1 commit intomasterfrom
feature/unify-to-date-converter
Jan 4, 2026
Merged

Unify _to_date converter and add git workflow guidelines#641
laughingman7743 merged 1 commit intomasterfrom
feature/unify-to-date-converter

Conversation

@laughingman7743
Copy link
Member

Summary

  • Extended base _to_date function to handle Union[str, datetime, date]
  • Removed duplicate _to_date implementations from arrow and polars converters
  • Added git workflow guidelines to CLAUDE.md

Changes

  • pyathena/converter.py: Extended _to_date to accept datetime and date objects
  • pyathena/arrow/converter.py: Removed duplicate _to_date, now imports from base
  • pyathena/polars/converter.py: Removed duplicate _to_date, now imports from base
  • CLAUDE.md: Added Git Workflow section with branch protection guidelines

Test plan

  • make chk passes (lint, format, type check)
  • CI tests pass

🤖 Generated with Claude Code

- Extended base _to_date function to handle Union[str, datetime, date]
- Removed duplicate _to_date implementations from arrow and polars converters
- Added git workflow guidelines to CLAUDE.md (never commit directly to master)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@laughingman7743 laughingman7743 marked this pull request as ready for review January 4, 2026 17:19
@laughingman7743 laughingman7743 merged commit f634cba into master Jan 4, 2026
5 checks passed
@laughingman7743 laughingman7743 deleted the feature/unify-to-date-converter branch January 4, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant